WIP: feat: create very simple wrappers #35

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Merged

Dam-Buty merged 17 commits into main from brendan/wrappers

Jul 10, 2024

Granipouss commented Jun 24, 2024

No description provided.


          feat: create very simple wrappers

20aa49f

desaxce reviewed

View reviewed changes

src/index.ts Outdated Show resolved Hide resolved

Granipouss and others added 3 commits

June 27, 2024 10:24


          test: add test to do


          new implementation

62c37a2


          merge main

f5c2d17

Dam-Buty marked this pull request as draft

July 4, 2024 11:50

desaxce reviewed

View reviewed changes

tests/wrappers.test.ts Outdated Show resolved Hide resolved

desaxce reviewed

View reviewed changes

src/index.ts Show resolved Hide resolved

desaxce reviewed

View reviewed changes

src/api.ts Outdated Show resolved Hide resolved

desaxce reviewed

View reviewed changes

src/index.ts Outdated Show resolved Hide resolved

desaxce reviewed

View reviewed changes

src/types.ts Outdated Show resolved Hide resolved

desaxce reviewed

View reviewed changes

tests/wrappers.test.ts Show resolved Hide resolved

Dam-Buty added 5 commits

July 4, 2024 17:19


          add tests for output changer

ed7274c


          before the great slashing

ce8f2fd


          continue experimenting

f23adf7


          remove experimental stuff

b4d6ab5


          move api tests

ceef8a2

Dam-Buty marked this pull request as ready for review

July 8, 2024 11:11

Dam-Buty added 3 commits

July 8, 2024 13:43


          minor stuff

f137ec2


          minor stuff

5a56e7c


          add tests on wrapping existing thread/step

f5f9aa7

desaxce approved these changes

View reviewed changes

Contributor

desaxce left a comment

Looks good to me

clementsirieix suggested changes

View reviewed changes

Contributor

clementsirieix left a comment

How do you feel about the breaking CI?

src/types.ts Show resolved Hide resolved

src/types.ts Outdated Show resolved Hide resolved

src/types.ts Show resolved Hide resolved

src/types.ts Outdated Show resolved Hide resolved

tsconfig.json Outdated Show resolved Hide resolved

Dam-Buty added 3 commits

July 8, 2024 19:07


          review fixes

5e03413


          fix attachment test

4ee3d98


          review fixes

a07850a

Dam-Buty requested review from willydouhard, clementsirieix and desaxce

July 8, 2024 17:23


          fix entity updating

300e152

clementsirieix approved these changes

View reviewed changes

Contributor

clementsirieix left a comment

LGTM


          feat: throw on no context

0789e01

willydouhard approved these changes

View reviewed changes

Contributor

willydouhard left a comment

lgtm

Dam-Buty merged commit 126bb52 into main

2 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet